Add Aqara T1 led strip quirk #4274
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## dev #4274 +/- ##
==========================================
+ Coverage 92.05% 92.16% +0.11%
==========================================
Files 350 359 +9
Lines 11601 11970 +369
==========================================
+ Hits 10679 11032 +353
- Misses 922 938 +16 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
TheJulianJES
changed the title
TheJulianJES
added
needs review
|
Just tested this to correct the length and it worked! I have one Aqara T1 LED Strip (2m) + 1 extension (1m). The length defaulted to 2, and I set it to 3. Now the whole strip lights up! |
I added also a couple of other supported options, for example, you can activate and configure audio mode, where lights changes in sync with sound. Hi @TheJulianJES, is there anything needed in this PR to get this in? |
|
Hi @stefankv93, I made a few changes to this PR at stefankv93#1, namely:
Please review and let me know if you have any questions or concerns. Thanks! |
Thanks for catching this, this is great! I included your commit changes in this PR. :) |
Proposed change
The repo is missing a quirk for Aqara T1 led strip light:
https://www.aqara.com/eu/product/led-strip-t1/
I recently got this device, hence got some time to develop a quirk for it.
Additional information
Fixes #2886
I tested the quirk with the custom quirk applied on my local home assistant instance:
Device diagnostics
Attaching device diagnostics with the quirk applied
zha-01JC6W4XYGYJJFR3D9AJJAQXJH-Aqara Led Strip T1-eb15b407c240b5e4f017fdf4f4798451.json
Checklist
pre-commitchecks pass / the code has been formatted using Black